Skip to content

ci: create a quality assurance workflow - #173

Merged
robertodr merged 84 commits into
mainfrom
ci-split-qa-jobs
Aug 3, 2026
Merged

ci: create a quality assurance workflow#173
robertodr merged 84 commits into
mainfrom
ci-split-qa-jobs

Conversation

@robertodr

@robertodr robertodr commented Jul 30, 2026

Copy link
Copy Markdown
Member

Summary

Separate quality assurance jobs from testing.

Changes

  • Split code coverage from test.yml to qa-analysis.yml
  • Split sonarqube from test.yml to qa-analysis.yml

Checklist

  • Tests added or updated to cover the changes
  • Documentation updated (docstrings, docs/, CONTRIBUTING.md) if needed
  • CHANGELOG / release notes updated if applicable

AI/LLM disclosure

  • I did not use LLM tooling, or used it only privately for ideation
  • I used the following tool to help write this PR description:
  • I used the following tool to generate or modify code:

Important

By opening this PR I confirm that I have read CONTRIBUTING.md and I agree to the terms of the Contributor License Agreement.

Warning

If you're contributing on behalf of your employer, contact cla@algorithmiq.fi to arrange a Corporate CLA.

@github-actions

Copy link
Copy Markdown

Docs preview: https://pr-173.monoprop-docs.pages.dev

@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.31%. Comparing base (fc1b9dc) to head (063e226).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #173   +/-   ##
=======================================
  Coverage   95.31%   95.31%           
=======================================
  Files          14       14           
  Lines         704      704           
  Branches       88       88           
=======================================
  Hits          671      671           
  Misses         20       20           
  Partials       13       13           
Flag Coverage Δ
cpp 95.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@robertodr
robertodr force-pushed the chore-dev-single-build-path branch from 2587fab to 802a56f Compare July 31, 2026 11:13
@robertodr
robertodr force-pushed the ci-split-qa-jobs branch 3 times, most recently from 818dabd to 6c119df Compare July 31, 2026 12:30
@robertodr
robertodr force-pushed the chore-dev-single-build-path branch from cffecae to 01cf9c8 Compare July 31, 2026 13:39
@robertodr
robertodr force-pushed the chore-dev-single-build-path branch from 01cf9c8 to 73da1b3 Compare July 31, 2026 17:35
robertodr and others added 15 commits August 1, 2026 19:44
The package is installed under `lib/python3.X/site-packages/monoprop` all other artifacts are installed in subfolders of it: * `libmonoprop.so` is under `lib` (or `lib64`, depending on the machine)
* all headers are under `include` with the proper hierarchical folder structure
* CMake helper files are under `cmake`  The header files are now explicitly listed (instead of globbed) and we use `target_sources` with `FILE_SET` pento handle them sanely (instead of appending to a global list)

I have dropped the distinction between private and public for now.
Signed-off-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com>
@robertodr
robertodr force-pushed the chore-dev-single-build-path branch from 88dfbf7 to 72965c2 Compare August 2, 2026 14:01
@robertodr

Copy link
Copy Markdown
Member Author

apologies: as you can see the history got completely mangled (34 commits, while it should be 7 or so) I'm learning how to use stacked PRs...

Base automatically changed from chore-dev-single-build-path to main August 3, 2026 11:11
Signed-off-by: Roberto Di Remigio Eikås <robertodr@users.noreply.github.com>
@sonarqubecloud

sonarqubecloud Bot commented Aug 3, 2026

Copy link
Copy Markdown

@robertodr
robertodr enabled auto-merge (squash) August 3, 2026 11:21
@robertodr
robertodr merged commit cf72f89 into main Aug 3, 2026
21 checks passed
@robertodr
robertodr deleted the ci-split-qa-jobs branch August 3, 2026 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants